Skip to content

Conversation

auouymous
Copy link
Member

No description provided.

@auouymous
Copy link
Member Author

@elelay For whatever reason, MAC_OS_X_VERSION_MIN_REQUIRED is not returning 101300 even though the runner is 13.7. So I patched out the checks in glib and librsvg. It might just work, or it could fail to link at runtime.

It appears Py_SetPath has been removed and Py_GetPath is deprecated and will be removed in 3.15. https://docs.python.org/3/c-api/init.html#c.Py_GetPath says to use sys.path instead. Do you know what that means? I assume we also use it to set the path.

https://docs.python.org/3/c-api/init.html#c.PySys_SetArgvEx has no removal date and isn't as important, yet.

misc/bundle/launcher.c:108:5: error: call to undeclared function 'Py_SetPath'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
    Py_SetPath(path);
    ^
misc/bundle/launcher.c:108:5: note: did you mean 'Py_GetPath'?
/Users/runner/work/gpodder-osx-bundle/gpodder-osx-bundle/_home/gtk/inst/include/python3.13/pylifecycle.h:46:43: note: 'Py_GetPath' declared here
Py_DEPRECATED(3.13) PyAPI_FUNC(wchar_t *) Py_GetPath(void);
                                          ^
misc/bundle/launcher.c:191:5: warning: 'PySys_SetArgvEx' is deprecated [-Wdeprecated-declarations]
    PySys_SetArgvEx(argc+1, wargv, 0);
    ^
/Users/runner/work/gpodder-osx-bundle/gpodder-osx-bundle/_home/gtk/inst/include/python3.13/sysmodule.h:11:1: note: 'PySys_SetArgvEx' has been explicitly marked deprecated here
Py_DEPRECATED(3.11) PyAPI_FUNC(void) PySys_SetArgvEx(int, wchar_t **, int);
^
/Users/runner/work/gpodder-osx-bundle/gpodder-osx-bundle/_home/gtk/inst/include/python3.13/pyport.h:251:54: note: expanded from macro 'Py_DEPRECATED'
#define Py_DEPRECATED(VERSION_UNUSED) __attribute__((__deprecated__))
                                                     ^

@elelay
Copy link
Member

elelay commented Jun 22, 2025

Hi @auouymous,
I've not had time to read it yet but https://peps.python.org/pep-0587/ is the way to go.
See python/cpython#93103

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants